v1.79.1.0 Make setup, sessions, and GBrain sync fail closed - #2758
Closed
armandovargash wants to merge 15 commits into
Closed
v1.79.1.0 Make setup, sessions, and GBrain sync fail closed#2758armandovargash wants to merge 15 commits into
armandovargash wants to merge 15 commits into
Conversation
|
Merging to
After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here |
Author
|
@codex review |
Author
Two hooks, both pull-mode, because gbrain's push lane needs a single-process PGLite engine and a shared brain cannot be one. session-start-hook puts the governing state in front of the agent before it guesses: the active goal, work that only exists on this disk, and the pointer to the factory map. On 2026-09-01 and twice on 2026-09-03 agents opened a session, read a stale file instead of the running system, and handed the CEO confident wrong diagnoses. The brain held the right answer every time and nothing made them look. skill-router-hook matches the user's own words against every skill's declared triggers on each message and names the skills that own the request. The routing contract already says to do this by hand; an agent that forgets to look routes on vibes. Triggers are declared in English, so the matcher also carries the Spanish phrasings this user actually types — "ponte al día", "se rompe", "vale la pena" — widening the haystack, never replacing his text. Both stay silent when nothing matches, never block a turn, and never spend on a model. They surface candidates; the agent still decides. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
docs/operations holds twenty-three memos and the open issues carry the decided design. On 2026-09-04 a session spent five hours rebuilding, from arxiv, a conclusion the 2026-07-08 coverage-ontology memo had already reached from the same sources: SBA, HBS/Sahlman, Business Model Canvas, NAICS, Steve Blank, Lean Startup, W3C PROV, FAIR, GOV.UK. The work was not missing. Nobody looked, because nothing said it was there. The start hook already named the factory map. It now also names the memo directory with its file count and tells the agent to search open issues before concluding a design is undecided. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Author
|
/trunk merge |
|
An error occurred while submitting your PR to the queue: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.